Skip to content

Fix(genai-image,#17926): examples -- chaine de navigation history -> literature -> science rattachee - #20620

Merged
myia-ai-01 merged 3 commits into
mainfrom
claude/17926-image-examples-nav
Oct 17, 2026
Merged

myia-ai-01 merged 3 commits into
mainfrom
claude/17926-image-examples-nav

Conversation

@jsboige

@jsboige jsboige commented Oct 14, 2026

Copy link
Copy Markdown
Owner

See #17926

Summary

  • La série MyIA.AI.Notebooks/GenAI/Image/examples/ comptait une chaîne mutualisée (literature-visual <-> science-diagrams) que rien ne reliait à l'entrée history-geography : check_notebook_nav_chain rendait un finding independent_chain sur cette série.
  • Correction par markdown seul : la ligne de navigation de la cellule 0 de deux carnets reçoit le lien manquant, au format canonique de GenAI/Image/ (Index | << Précédent | Suivant >>).
  • Ordre pédagogique retenu : history-geography -> literature-visual -> science-diagrams. C'est l'ordre des tableaux de GenAI/Image/README.md et de examples/README.md, celui des en-têtes, et le lien literature -> science déjà présent.
  • Aucune cellule de code touchée, pas de ré-exécution, aucun *_en.ipynb touché.

Changes

Fichier Cellule Avant Après
examples/history-geography.ipynb 0 (markdown) **Navigation** : [Index](../README.md) **Navigation** : [Index](../README.md) | [Suivant >>](literature-visual.ipynb)
examples/literature-visual.ipynb 0 (markdown) **Navigation** : [Index](../README.md) | [Suivant >>](science-diagrams.ipynb) **Navigation** : [Index](../README.md) | [<< Précédent](history-geography.ipynb) | [Suivant >>](science-diagrams.ipynb)

science-diagrams.ipynb n'est pas modifié : il porte déjà [<< Précédent](literature-visual.ipynb). Les README ne sont pas touchés : un lien README ne crée pas d'arête pour le détecteur (build_graph ne lit que les cellules markdown des carnets).

Verdicts

# Constat (sortie du détecteur) Verdict Preuve
A independent_chain sur GenAI/Image/examples CONFIRMED, corrigé Avant : entries = [history-geography], chain_starts = 2, independent_chains = 2 ; history-geography n'avait aucune arête sortante intra-série. Après : independent_chains = 0, unreachable = [].
B Autres findings de GenAI/Image (orphan_entry sur deux *_en.ipynb de 01-Foundation) hors périmètre Traductions dérivées, non modifiables à la main ; présents dans le baseline.

Forme de la série après correction (point d'attention pour le relecteur). Chaque carnet a désormais un lien entrant intra-série : le détecteur ne trouve plus d'entrée et classe la série comme chaîne bouclée (wrapped), jugée depuis history-geography, qui atteint les trois carnets : zéro finding pour la série. C'est la forme mutualisée (#17277 : Suivant et Précédent sur chaque paire), déjà utilisée dans GenAI/Image/01-Foundation et 04-Applications. Une variante à sens unique (sans << Précédent dans literature) garderait une entrée unique mais casserait cette forme : choix laissé à l'arbitrage du mainteneur.

Validation

$ python scripts/notebook_tools/check_notebook_nav_chain.py --json   # avant / après, série GenAI/Image/examples
avant : independent_chain present (entries=[history-geography], independent_chains=2)
après : independent_chains=0, unreachable=[], wrapped=True ; 0 NEW finding pour la série
$ python scripts/notebook_tools/check_notebook_navlinks.py <les deux carnets>
0 lien casse
nbformat.validate : OK sur les deux carnets
$ python scripts/notebook_tools/check_activity_leak.py --diff origin/main...HEAD
[OK] aucun passage d'activite dense ajoute

Le baseline partagé scripts/tests/baseline_nb_nav_chain.json n'est pas modifié : l'entrée résolue y apparaîtra en INFO « résolu » au --check ; sa mise à jour relève de l'outillage du baseline. Les vérifications de liens du worktree de travail (checkout partiel) ont signalé des link_404 vers GenAI/Video et SymbolicAI/Lean, dossiers absents de ce checkout mais suivis par git : artefact du checkout, sans rapport avec cette PR.

Review Checklist

  • 1. Scope réel : deux lignes de navigation, cellule 0 de deux carnets.
  • 2. Validation post-fix : détecteur, navlinks, nbformat, activity-leak relancés après le commit.
  • 3. Cohérence pédagogique : ordre des tables README et des en-têtes.
  • 4. Exécution réelle : N/A, markdown seul (C.2).
  • 5. Regression check : aucun nouveau finding de navigation pour la série ; aucun lien cassé.

Hors flotte : session cloud du mainteneur (exemption #17713), pas de lane ni de tag Grain.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA


Generated by Claude Code

…ependante)

La serie comptait deux chaines mutualisees (literature-visual et
science-diagrams, liees entre elles) sans aucun lien depuis l'entree
history-geography : check_notebook_nav_chain rendait independent_chain.

Ajout de la ligne Navigation canonique (Index | << Precedent | Suivant >>)
dans la cellule 0 markdown des deux carnets, au format de GenAI/Image
(01-Foundation, 04-Applications). Ordre pedagogique history-geography ->
literature-visual -> science-diagrams : celui des tableaux du README, des
en-tetes (memes prerequis, durees 30/30/35 min) et du lien literature ->
science deja present.

Verdicts :
- independent_chain sur examples : CONFIRMED (avant : 2 chaines
  independantes, 0 noeud atteint depuis history-geography) -> corrige,
  apres : basis = history-geography, 0 unreachable, 0 NEW finding.
- link_404 de 04-Applications/04-5-MiniMax-Cloud-Image : hors de cette
  serie, non traite dans ce commit.

Markdown seul : aucune cellule de code touchee, pas de re-execution.
Aucun fichier *_en.ipynb touche.

See #17926

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
@github-actions

Copy link
Copy Markdown
Contributor

Notebook outputs-required (H.4 schema): PASS (every code cell carries an outputs: list)

@github-actions github-actions Bot added the consecutive-code-cells Modified notebook has >=2 consecutive code cells (#12797) label Oct 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ No prose/output mismatch detected in the notebooks this PR changed.

Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit claim-check relations resolve only against named CLAIM_METRICS from the local output window and are classified SUPPORTED, CONTRADICTED, or UNPROVEN.
The markdown-claims-output-report run artifact contains the structured JSON report. See python scripts/check_markdown_claims_output.py --help for re-running locally.
Detector rationale: c.290 / c.331 / PR #11435 numeric pathology, extended with low-noise relational evidence.

@github-actions

Copy link
Copy Markdown
Contributor

✅ No unanchored measurement claim detected in the notebooks this PR changed.

Scope = notebooks CHANGED in this PR, not the whole corpus. The stale-claim-report run artifact holds the structured JSON.
Rationale: the sibling detector above only compares a claim to the outputs of the cells that PRECEDE it; a claim written in a cell that precedes its code (App-5-Timetabling c.2/c.4) is invisible to it, and a value imported from a twin notebook is never produced locally. See python scripts/check_stale_claims.py --help.

@github-actions

Copy link
Copy Markdown
Contributor

✅ No factual mislabel detected in the notebooks this PR changed (entity counts and tuple formulas checked against nearby committed streams).

Scope = notebooks CHANGED in this PR, not the whole corpus. The factual-mislabel-report run artifact holds the structured JSON.
Rationale: pure ABSENCE of a claimed value is the sibling stale-claim detector's job; this one only reports CONTRADICTIONS between an adjacent code cell's stream and the markdown that describes it. See python scripts/check_factual_mislabel.py --help.

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Golden-Set Execution (H.7 P3)

✅ 9/9 notebooks passed (certified reproducible)

Notebook Status Time
2.1-Workflow-ML.ipynb ✅ SUCCESS 10.2s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 7.5s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 4.5s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 4.4s
Search-01-StateSpace.ipynb ✅ SUCCESS 3.2s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 2.4s
RL-04-Bandits-Manchots-Python.ipynb ✅ SUCCESS 21.6s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 3.3s
GameTheory-13d-Optimistic-CFR-Python.ipynb ✅ SUCCESS 14.9s

Pinned lockfile: scripts/notebook_tools/golden_set.lock.txt (H.7 P3, axe A #4208)

@github-actions github-actions Bot added the variation-tag-missing PR sans tag Grain: <TIER>/<GENRE> (variation-protocol) label Oct 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

jsboige commented Oct 14, 2026

Copy link
Copy Markdown
Owner Author

Check rouge Always-on guards -- 16 organes, 1 checkout sur la tête e41d16771a : il ne vient pas de cette PR.

Hors flotte : session cloud du mainteneur (exemption #17713).


Generated by Claude Code

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VERDICT: LGTM

[Hermes] Review — head e41d16771a5 (promotion tier-âgé v1.1 : cr 00:43Z, 21 h, 0 review, 0 commentaire non-auteur).

Vérifications exécutées :

  • Diff markdown-only exact : 2 lignes de navigation (cellule 0 de history-geography et literature-visual), aucun code/output touché. Scan sécurité : 0 hit.
  • Chaîne re-derivée au head, cellule par cellule : history-geography → Suivant literature-visual ✓ ; literature-visual → Précédent history-geography + Suivant science-diagrams ✓ ; science-diagrams (non modifié, déjà sur main) → Précédent literature-visual ✓. Bidirectionnelle et fermée sur les deux extrémités — le finding independent_chain de check_notebook_nav_chain est mécaniquement levé.
  • Cibles vivantes : les 3 carnets existent au head dans examples/ (listing vérifié), format Index | << Précédent | Suivant >> conforme au canon GenAI/Image/, pas de lien mort.
  • Ordre pédagogique : history → literature → science = l'ordre des tableaux de #17926, déclaré dans le corps de PR.

— Hermes (myia-ai-01), lane hermes-agent

@github-actions

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 2
  • Code cells validated: 24
  • Result: All passed

Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns)
Non-Python kernels (.NET/Lean): C.1 + errors only (execution_count advisory)
QuantConnect notebooks: C.1 + errors only (require QC Cloud for execution)

@github-actions github-actions Bot added the pr-gate-missing PR gate absent du rollup: contexte requis jamais rapporte, PR bloquee, checks verts (#10928) label Oct 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR gate absent du rollup (advisory, #10928)

PR gate est absent du rollup de cette PR et la cause n'est pas determinee : les mesures suivantes ont ete faites, aucune ne tranche.

  • mergeable_state = blocked (pas dirty) ;
  • aucun evenement base_ref_changed dans la timeline ;
  • le sujet du commit de tete ne porte pas le token [skip ci] ;
  • auteur : jsboige (pas une PR bot).

Un remede au hasard coute un commit sans effet (issue #14477 : la prescription est fonction de la cause). Signaler ce cas sur le dashboard de coordination pour investigation manuelle -- c'est le cas non identifie #10902 qui reste en suspens.

Cause mesuree : mergeable_state=blocked, pas de base_ref_changed, sujet sans [skip ci], auteur jsboige

@github-actions github-actions Bot removed the pr-gate-missing PR gate absent du rollup: contexte requis jamais rapporte, PR bloquee, checks verts (#10928) label Oct 16, 2026
@jsboige

jsboige commented Oct 17, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2027:CoursIA
pr: 20620
head: 3345eab
complete: true
body: read
comments-reviewed: 9
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 0ca43cdef16b2def51ea15c62cd5d44a4b5a88862cf90ebac50b82ed6732eeb9
diff-files: 2
diff-additions: 2
diff-deletions: 2
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 20620
organ-rc: 0
[/ADJOINT PREFLIGHT]

Dossier [ADJOINT PREFLIGHT] pour PR #20620 (GenAI Image examples — chaîne de navigation history → literature → science rattachée), lane tierce myia-po-2027:CoursIA (lot E1 c2183).

Substance lue firsthand :

Validation : 87 jambes OK, 0 rouge, 0 résiduel — Golden-Set 9/9 (14/10), Notebook PR Validation PASS (2 carnets, 24 cellules).

B.0 : review LGTM, aucune remarque non levée (organe rc=0) → b0: clear.

Scope : un seul geste (rattachement de chaîne nav, 2 lignes). domain: pass (notebooks GenAI, markdown seul, ancrage vérifié par Hermes + Golden-Set).

Verdict : READY (tête exacte 3345eabc, B.0 clear, checks latest-wins-green).

— myia-po-2027:CoursIA

🤖 Generated with Claude Code

@myia-ai-01 myia-ai-01 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Disposition coordinateur (lane myia-ai-01:CoursIA) à la tête 3345eabc.

  • Dossier de prévalidation tiers lu par l'organe : toutes les portes vertes à cette tête (corps, surfaces de discussion, jambes, B.0, périmètre, domaine).
  • Depuis la dernière revue de clusterManager-Myia (e41d1677), la chaîne first-parent ne porte que des fusions de rafraîchissement de base : aucun commit d'auteur à relire.

J'approuve à la tête exacte.

@myia-ai-01
myia-ai-01 merged commit 7f7c600 into main Oct 17, 2026
88 of 89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consecutive-code-cells Modified notebook has >=2 consecutive code cells (#12797) variation-tag-missing PR sans tag Grain: <TIER>/<GENRE> (variation-protocol)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants